The Arduino function bitSet() is actually a macro and it turns into sbi and cbi by the compiler.
The sbi and cbi in a 16MHz Arduino are 125 ns.
And the digitalRead() and digitalWrite() are between 3 and 4 us.
You must use your own pin definitions, like _D13 because you store all the information about the port into it.
That is very clever, but I don't know what it will do in the Arduino environment.
How would setting the output HIGH and LOW be ?
What if the number of the pin is a variable ?