[llvm-commits] [llvm] r130544 - /llvm/trunk/lib/MC/MCDwarf.cpp

[Anton Korobeynikov]

Hi Rafael,

> more importantly lets this code be used when producing assembly code for old assemblers without
> uleb support.
I don't think it's a good idea in general. In such a way we can not
emit .cfi stuff at all, because "old assemblers don't support it".
The change definitely makes the code less readable, because it emits
some magic constants instead of direct label difference.

Maybe the stuff can be conditionalized? E.g. via "HasUleb" option or
something like this?

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University