================
@@ -2642,12 +2642,80 @@ static RValue
EmitHipStdParUnsupportedBuiltin(CodeGenFunction *CGF,
return RValue::get(CGF->Builder.CreateCall(UBF, Args));
}
+/// Determines whether we should treat a given built-in as a call to an
+/// artificial inlined function in debug info.
+static bool shouldUseBuiltinDebugLocation(unsigned BuiltinID) {
----------------
jryans wrote:
I took another look at this... I currently remain convinced that it is easier
for myself (and hopefully other readers also) to follow the exceptions list in
its current `switch` statement form. It's easier to see the several classes of
exceptions that way, and also easier to add comments explaining the reasoning
for each. (There are many other parts of Clang that switch on builtin ID in a
similar manner, so it doesn't feel too out of place in my opinion.)
Of course, if someone does feel strongly that it really should be in
`Builtins.td`, we can still explore that.
https://github.com/llvm/llvm-project/pull/189969
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
