[PATCH] Use function attribute "arm-restrict-it"

Renato Golin renato.golin at linaro.org
Fri Jun 19 04:28:13 PDT 2015


I'm confused. You removed the back-end option in Clang, but it remains here (IT), meaning you now have two identical command line options that do things slightly different. The function attributes set IT behaviour on every function and the backend option that sets it in a module level.

This will create a confusing environment on which to develop things and I don't want to extend this pattern. If the option needs to be on a function basis, than we need to make sure that whatever generates IR does that on a function level, and free the backend from having to ponder about module level options.

In summary, IT options must go. :)

cheers,
--renato


http://reviews.llvm.org/D10416

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list